Skip to content

Engine deps on @bevyl-ai/agent-tools (kit rotation + gateway config)#42

Merged
Octember merged 3 commits into
mainfrom
agent-tools-dep
Jul 11, 2026
Merged

Engine deps on @bevyl-ai/agent-tools (kit rotation + gateway config)#42
Octember merged 3 commits into
mainfrom
agent-tools-dep

Conversation

@Octember

Copy link
Copy Markdown
Owner

stupify now consumes @bevyl-ai/agent-tools directly instead of carrying an inline copy of the rotation mechanism. The engine stays a single dependency-free VM artifact because installSweepEngine bundles via Bun.build — the kit inlines at install time (verified: the built bundle has zero external imports).

  • Rotation moves into reviewPr's limit branch where the real codex failure text is in scope, using the kit's maybeRotateGateway with pool/cooldown fed from config.env (new pool/cooldownMs opts in 0.2.0). Behavior tightening, deliberate: the kit's quota-wall signature is stricter than isRateLimited, so a transient 429 still ends the sweep early but no longer advances the gateway ring.
  • The CLI adopts the kit's writeCodexGatewayConfig (with trustDir); @stupify/exe-cli drops its copy.
  • Rotation mechanics tests live with the code in the kit now.

Companions: bunion#73 (kit ssh helpers), earshot#4 (kit scrubSecrets).

bun.lock refresh pending the 0.2.0 npm publish (OTP).

Octember added 2 commits July 11, 2026 12:21
The engine is bundled by Bun.build at install time, so the kit dep is
inlined into the single deployed file — the dependency-free VM artifact
survives (verified: bundle has zero external imports). Changes:
- review-sweep: drop the inline rotateGateway; rotation moves into
  reviewPr's limit branch where the REAL failure text is in scope, via the
  kit's maybeRotateGateway (pool/cooldown from config.env via new opts).
  The kit's signature match is tighter than isRateLimited by design: a
  transient 429 still ends the sweep but no longer walks the ring.
- cli: adopt the kit's writeCodexGatewayConfig (trustDir opt); exe-cli
  drops its copy.
- rotation mechanics tests move to the kit with the code.

bun.lock refresh pending the 0.2.0 npm publish.

@exe-dev-github-integration exe-dev-github-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hm, one real issue: this adds a real runtime/typecheck dependency without making installs reproducible.

Comment thread package.json

@exe-dev-github-integration exe-dev-github-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hm, one real issue remains: the rotation path lost the raw failure text it says it needs.

Comment thread src/review-sweep.ts Outdated
failureReason() picks one signal line and truncates to 220 chars for the
log; feeding that to the kit's stricter isQuotaWall could silently skip a
real rotation. Carry the raw output on the limit result and match on that.
(Reviewer catch on #42.)

@exe-dev-github-integration exe-dev-github-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice, all fixed ✅

@Octember Octember merged commit 9f9454b into main Jul 11, 2026
2 checks passed
@Octember Octember deleted the agent-tools-dep branch July 11, 2026 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant